Skip to content

Conversation

@AoElite
Copy link

@AoElite AoElite commented Nov 25, 2024

Bump minimum velocity version to 21. It appears intentional that Velocity is ran with at least Java 21 since parts of the code base rely on specific Java 21 features. Such as commit PaperMC/Velocity@ecf936f.

Bump minimum velocity version to 21
@github-actions
Copy link

github-actions bot commented Nov 25, 2024

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
papermc-docs ✅ Ready (View Log) Visit Preview c054bdd

@zlataovce
Copy link
Member

that commit looks like it was made to be compatible with versions <21 and the defined toolchain is still Java 17, so I'd appreciate it if someone from the Velocity team could weigh in here (cc @4drian3d)

maybe it'd be worth to specify a different recommended version beside the minimum version

@olijeffers0n olijeffers0n added the help wanted Additional input is requested label Nov 25, 2024
@zlataovce zlataovce added the pre-migration Pull requests which point to a commit before the Astro migration label Apr 20, 2025
zlataovce added a commit that referenced this pull request Apr 20, 2025
@zlataovce zlataovce closed this Apr 20, 2025
sowelipililimute pushed a commit to sowelipililimute/docs that referenced this pull request Sep 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

help wanted Additional input is requested pre-migration Pull requests which point to a commit before the Astro migration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants